Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(applepay): calculate tax amount dynamically while changing shipping address in apple pay sdk #610

Merged
merged 3 commits into from
Sep 13, 2024

Conversation

ImSagnik007
Copy link
Contributor

@ImSagnik007 ImSagnik007 commented Sep 12, 2024

Calculate tax amount and shipping cost dynamically while changing shipping address in apple pay sdk and update the new amount.

Listening to the onShippingContactSelected listener while there is a change in shipping address, abd based on the new shipping address calling taxjar api to get new amount and updating that in sdk.

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

How did you test it?

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

Copy link

Review changes with SemanticDiff.

…ing address in apple pay sdk

Calculate tax amount and shipping cost dynamically while changing shipping address in apple pay sdk
and update the new amount.
seekshiva
seekshiva previously approved these changes Sep 13, 2024
@PritishBudhiraja PritishBudhiraja added the Ready for Review PR with label Ready for Review should only be reviewed. label Sep 13, 2024
@seekshiva seekshiva merged commit 0804823 into main Sep 13, 2024
3 checks passed
@seekshiva seekshiva deleted the tax_calculation_support_apple_pay branch September 13, 2024 08:47
@github-actions github-actions bot added Closed Label will be automatically added when the PR will get merged to main and removed Ready for Review PR with label Ready for Review should only be reviewed. labels Sep 13, 2024
akash-c-k pushed a commit that referenced this pull request Sep 13, 2024
# [0.86.0](v0.85.0...v0.86.0) (2024-09-13)

### Features

* **applepay:** calculate tax amount dynamically while changing shipping address in apple pay sdk ([#610](#610)) ([0804823](0804823))
@@ -83,6 +83,7 @@ type eventName =
| DELETE_SAVED_PAYMENT_METHOD
| DELETE_PAYMENT_METHODS_CALL_INIT
| DELETE_PAYMENT_METHODS_CALL
| EXTERNAL_TAX_CALCULATION
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

| EXTERNAL_TAX_CALCULATION_CALL
| EXTERNAL_TAX_CALCULATION_CALL_INIT

@@ -166,6 +167,7 @@ let eventNameToStrMapper = eventName => {
| DELETE_SAVED_PAYMENT_METHOD => "DELETE_SAVED_PAYMENT_METHOD"
| DELETE_PAYMENT_METHODS_CALL_INIT => "DELETE_PAYMENT_METHODS_CALL_INIT"
| DELETE_PAYMENT_METHODS_CALL => "DELETE_PAYMENT_METHODS_CALL"
| EXTERNAL_TAX_CALCULATION => "EXTERNAL_TAX_CALCULATION"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

|EXTERNAL_TAX_CALCULATION_CALL=>"EXTERNAL_TAX_CALCULATION_CALL"
|EXTERNAL_TAX_CALCULATION_CALL_INIT=>"EXTERNAL_TAX_CALCULATION_CALL_INIT"

~optLogger=Some(logger),
~url=uri,
~apiLogType=Request,
~eventName=EXTERNAL_TAX_CALCULATION,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~eventName=EXTERNAL_TAX_CALCULATION_CALL_INIT

~data,
~statusCode,
~apiLogType=Err,
~eventName=EXTERNAL_TAX_CALCULATION,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~eventName=EXTERNAL_TAX_CALCULATION_CALL

~url=uri,
~statusCode,
~apiLogType=Response,
~eventName=EXTERNAL_TAX_CALCULATION,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~eventName=EXTERNAL_TAX_CALCULATION_CALL

~optLogger=Some(logger),
~url=uri,
~apiLogType=NoResponse,
~eventName=EXTERNAL_TAX_CALCULATION,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~eventName=EXTERNAL_TAX_CALCULATION_CALL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Label will be automatically added when the PR will get merged to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Add support for dynamic tax and shipping cost calculation in apple pay
5 participants